Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make it possible for users to view their data on Panorama Public #387

Merged
merged 7 commits into from
Jan 21, 2024

Conversation

vagisha
Copy link
Collaborator

@vagisha vagisha commented Jan 8, 2024

Changes

  • Added a "My Data" button in the "Panorama Public Experiments" button bar. The button is displayed if a user is logged in. It links to MyDataViewAction.
    image

  • My Data view displays a grid filtered to the user's public and private datasets on Panorama Public. These are experiments where the user is either a "submitter" or the "lab head".

  • The "My Data" grid displays a "Catalog Entry" column that can be used to add or view catalog entries for public datasets. Catalog entries cannot be added for private datasets.

image

@vagisha vagisha requested a review from labkey-jeckels January 8, 2024 03:57
}
}

public static class MyDataForm
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this form at all? It seems like we only support viewing data for the current user.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, we don't need a userId since we are only displaying the data for the currently logged in user! I removed the form.

return;
}
}
HtmlString.EMPTY_STRING.appendTo(out);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This codepath uses EMPTY_STRING and the one above uses NBSP. I imagine they should probably use the same placeholder, though maybe it doesn't make a difference.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

…g.NBSP consistently)

- Use return URL, when available, for rendering the back button in the CatalogEntryWebPart.
@vagisha vagisha merged commit 85aecb9 into release23.11-SNAPSHOT Jan 21, 2024
3 checks passed
@vagisha vagisha deleted the 23.11_fb_my-data-view branch January 21, 2024 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants